Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FEATURE: New and Unread messages for user personal messages. #13603

Merged
merged 3 commits into from Aug 2, 2021

Conversation

tgxworld
Copy link
Contributor

@tgxworld tgxworld commented Jul 1, 2021

This PR introduces the "Unread" and "New" filter when viewing personal messages. However, the additional of the two new filters means that we have to revamp the UI of personal messages so that we do not display a ton of options in the navigation links. In the UI revamp, we allow messages to be filtered by the different inboxes as well as tags.

Screenshots

Desktop with inboxes filter

Screenshot from 2021-07-22 11-32-57

Desktop without inboxes filter

Screenshot from 2021-07-26 14-44-55

Mobile with inboxes filter

Screenshot from 2021-07-22 11-34-12

Mobile without inboxes filter

Screenshot from 2021-07-26 14-44-09

Group Inbox

Screenshot from 2021-07-22 11-33-15

Things that have been left out intentionally in this PR

  1. MessageBus updates for "Unread" and "New" messages
  2. Searching for tags in the tags dropdown

@tgxworld tgxworld force-pushed the unread_new_messages branch 7 times, most recently from 4933128 to a5dd25d Compare July 8, 2021 08:30
@lgtm-com
Copy link

lgtm-com bot commented Jul 8, 2021

This pull request introduces 1 alert when merging a5dd25d4e09fc87d579decf04a94624b858741ac into 87c1e98 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@tgxworld tgxworld force-pushed the unread_new_messages branch 22 times, most recently from d3dfe7a to 01aef27 Compare July 22, 2021 03:14
@tgxworld tgxworld force-pushed the unread_new_messages branch 22 times, most recently from 84da8a9 to 96fd45a Compare July 29, 2021 07:53
@tgxworld tgxworld merged commit 016efea into discourse:main Aug 2, 2021
@tgxworld tgxworld deleted the unread_new_messages branch August 2, 2021 04:41
@discoursebot
Copy link

This pull request has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/new-all-pms-feature-timing-out/203002/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
6 participants